-
-
Notifications
You must be signed in to change notification settings - Fork 29
2.0.0-beta #1142
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
2.0.0-beta #1142
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR bumps the version to 2.0.0-beta while refactoring build configurations and updating dependencies across multiple packages. Key changes include:
- Updating tsup configurations to use ESM only and targeting Node 20 instead of Node 18.
- Bumping package.json versions and dependency versions (e.g. @typescript-eslint/* packages) consistently.
- Migrating internal type references (e.g. from "./ast-node" to "./ast-node-types") and deprecating legacy rule files.
Reviewed Changes
Copilot reviewed 223 out of 257 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| packages/utilities/var/tsup.config.ts | Build config updated to ESM only and Node20 target. |
| packages/shared/package.json | Version bumped and Node engine updated to >=20.19.0. |
| packages/utilities/ast/src/ast-node-format.ts | Renamed function "toDelimiterCaseType" to "toDelimiterFormat" for clarity. |
| packages/plugins/eslint-plugin-react-x (various files) | Refactored rule implementations and updated deprecation/replacement info. |
Comments suppressed due to low confidence (2)
packages/utilities/var/tsup.config.ts:8
- Removing CommonJS output in favor of ESM-only distribution may impact consumers relying on CommonJS; verify that downstream applications are compatible with ESM.
format: ["esm"],
packages/utilities/ast/src/ast-node-format.ts:17
- Ensure that the renaming from 'toDelimiterCaseType' to 'toDelimiterFormat' is reflected in all references across the codebase to prevent potential reference errors.
export function toDelimiterFormat(node: TSESTree.Node, delimiter = " ") {
What kind of change does this PR introduce?
Does this PR introduce a breaking change?
Checklist
fix: remove a typo, closes #___, #___)Other information